feat: ENG-12261 add OAuth 2.0 support and settings UI for Phrase integration#4699
feat: ENG-12261 add OAuth 2.0 support and settings UI for Phrase integration#4699sanyamkamat wants to merge 23 commits into
Conversation
|
|
View your CI Pipeline Execution ↗ for commit 854dc46
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
…nnection and disconnection functionality
…ion disconnection state
…leakage across organizations
…for improved isolation across organizations
…ization-specific plugin settings
…ons for Phrase integration
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 962ef5e. Configure here.

Description
Adds OAuth 2.0 support and settings UI for Phrase integration
Note
Medium Risk
Touches authentication and all Phrase translation API calls via a new client path; OAuth/session edge cases (org switch, stale metadata, token expiry) need correct behavior in production.
Overview
Adds SSO / OAuth 2.0 as an alternative to username/password for the Phrase connector, with settings for auth mode, per-org OAuth Client ID, and a Connect / Disconnect panel that runs the Builder API popup flow (
prepare→start→postMessage). Access tokens stay on the server; the browser only tracks connection metadata plus apiKey-scoped session markers for connect/disconnect before org settings reload.Replaces the direct
Phraseclient withPhraseApi, which calls/api/v1/memsource/*with the plugin private key, runsensureAuthenticatedfor Translate/Apply, and retries afteroauth/refreshon 401 in OAuth mode. Username/password fields are hidden when OAuth is selected; both credential types can coexist when switching modes.README documents OAuth setup (redirect URI, US data center, sequence diagram). Package version bumps to 0.0.17-1; the stub Request an updated translation content action is removed while translation freshness checking is refactored into
checkTranslationFreshness.Reviewed by Cursor Bugbot for commit 7abd3c7. Bugbot is set up for automated code reviews on this repo. Configure here.